-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open AI panel when AI app generator starts generating #1989
Open AI panel when AI app generator starts generating #1989
Conversation
@@ -622,7 +623,7 @@ module('Acceptance | Commands tests', function (hooks) { | |||
assert.dom('[data-test-card-url-bar-input]').hasValue(`${testCard}.json`); | |||
await click('[data-test-submode-switcher] button'); | |||
await click('[data-test-boxel-menu-item-text="Interact"]'); | |||
await click('[data-test-open-ai-assistant]'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The behavior changed now that I moved the aiAssistantOpen
from the component to the state service–previously, if the AI panel was open and the user toggled from interactive to code mode and then back to interactive, the AI panel was closed. But now, the AI panel remains open when going back to interactive mode. I think this behavior is actually more desirable, let me know if you think otherwise.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of introducing a new action on context.actions, we should introduce a new host command.
I added a new command for the purpose of opening an AI assistant room. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work
This PR adjusts the following:
experimentalAIEnabled
flag - I believe this is not experimental anymorecurrentRoomId
andaiAssistantOpen
from component state to matrix and operator mode state services so that we can manipulate them from other parts of the appopenRoom
commandHere in this video you can see the AI panel opens when user clicks on "Let's get started":
Screen.Recording.2024-12-27.at.13.24.06.mov